Skip to content

Added constructors#6

Open
roeman wants to merge 2 commits into
jenkinsci:masterfrom
roeman:master
Open

Added constructors#6
roeman wants to merge 2 commits into
jenkinsci:masterfrom
roeman:master

Conversation

@roeman

@roeman roeman commented Jun 28, 2017

Copy link
Copy Markdown

This seems to fix the issue with Python scripts not being available for use in conditional build steps.

@roeman roeman closed this Jun 28, 2017
@roeman

roeman commented Jun 28, 2017

Copy link
Copy Markdown
Author

I see someone else submitted a change for fixing this issue, but mine didn't seem to work without this extra code?

@roeman roeman reopened this Jun 28, 2017
@elitenas

elitenas commented Apr 13, 2021

Copy link
Copy Markdown

@olivergondza any way to merge this in? This change is much needed and highly requested

@olivergondza

Copy link
Copy Markdown
Member

@mazimkhan, as a maintainer, please take a look.

@elitenas

Copy link
Copy Markdown

Hi @mazimkhan,

Please take a look at these conversations...

https://issues.jenkins.io/browse/JENKINS-12483
https://issues.jenkins.io/browse/JENKINS-40800

According to this web page (https://wiki.jenkins-ci.org/display/JENKINS/Conditional+BuildStep+Plugin), the problem is because the Python Plugin "does not provide a @DataBoundConstructor constructor and/or the Descriptor does not extend hudson.tasks.BuildStepDescriptor."

@elitenas

Copy link
Copy Markdown

@olivergondza, is there any one else who has the ability to merge this PR?

@mazimkhan

Copy link
Copy Markdown
Contributor

Apologies for this late reply. I don't see the option to merge this PR. Not sure it this is due to failing CI.

Besides, I no longer use Jenkins or maintain this plugin. If someone else wants to take over, please do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants